Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow more time for restart tests to reach yellow state. #48434

Merged
merged 1 commit into from
Oct 24, 2019

Conversation

jakelandis
Copy link
Contributor

The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.


As part of re-enabling via #47950 this test was run for 24+ hours without failure on a dedicated node. I believe that this timeout is an environmental issue and 60s should be plenty of time.

The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
@jakelandis jakelandis added >test Issues or PRs that are addressing/adding tests :Data Management/Watcher v8.0.0 v7.6.0 labels Oct 23, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Watcher)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jakelandis jakelandis merged commit 28bfa6e into elastic:master Oct 24, 2019
@jakelandis jakelandis deleted the increase_timeout_for_yellow branch October 24, 2019 13:50
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Oct 24, 2019
The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Oct 24, 2019
The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
jakelandis added a commit that referenced this pull request Oct 24, 2019
…8480)

The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Nov 4, 2019
…r yellow

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: elastic#48381
Related: elastic#48434
Related: elastic#47950
Related: elastic#40178
jakelandis added a commit that referenced this pull request Nov 4, 2019
The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: #48381
Related: #48434
Related: #47950
Related: #40178
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Nov 4, 2019
…tic#48848)

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: elastic#48381
Related: elastic#48434
Related: elastic#47950
Related: elastic#40178
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Nov 4, 2019
…tic#48848)

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: elastic#48381
Related: elastic#48434
Related: elastic#47950
Related: elastic#40178
jakelandis added a commit that referenced this pull request Nov 11, 2019
The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: #48381
Related: #48434
Related: #47950
Related: #40178
jakelandis added a commit that referenced this pull request Nov 11, 2019
The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: #48381
Related: #48434
Related: #47950
Related: #40178
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Watcher >test Issues or PRs that are addressing/adding tests v7.5.0 v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants